Skip to content

Added posible fix for pipeline#260

Closed
sander-hash wants to merge 2 commits into
smi2:masterfrom
sander-hash:fix/pipeline-clickhouse-error
Closed

Added posible fix for pipeline#260
sander-hash wants to merge 2 commits into
smi2:masterfrom
sander-hash:fix/pipeline-clickhouse-error

Conversation

@sander-hash

Copy link
Copy Markdown
Contributor

No description provided.

@sander-hash sander-hash marked this pull request as draft April 29, 2026 16:01
@isublimity

Copy link
Copy Markdown
Contributor

Thank you for taking the time to look into the CI pipeline and for the proposed improvement! 🙏

Closing this one as the CI workflow has since been reworked directly on master and now covers the same ground:

  • the "copy config" step already does a docker restart + a /ping polling loop (instead of sleep);
  • a ClickHouse Keeper config was added so ON CLUSTER tests can run on the single-node image.

One important note for the future: the ClickHouse server images do not ship curl — only wget. The health-check/poll here uses curl, which was actually the root cause of the CI being red since April (the service container never became healthy). On master the health-check now uses wget --spider.

Really appreciate the contribution and the investigation — thank you! 🦫

@isublimity isublimity closed this Jun 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants